-
Notifications
You must be signed in to change notification settings - Fork 125
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Relaxation Runge-Kutta Methods #298
Conversation
Conflicts: examples/arkode/C_serial/CMakeLists.txt
I think I've address most of the PR comments and have left unresolved those that might need further discussion. |
I've addressed the remaining comments, this should be ready for another review pass. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggested a fix for ark_pendulum.cpp
to improve the output (as noted in the accompanying "answers" PR). With this suggestion, I'm happy with both PRs.
Add support for explicit, implicit, and IMEX relaxation Runge-Kutta methods in ERKStep and ARKStep